Skip to content

[17.0] [ADD] mail_discuss_channel_unread_sort: New module#1827

Open
miquelalzanillas wants to merge 2 commits intoOCA:17.0from
APSL:17.0-add-mail_discuss_channel_unread_sort
Open

[17.0] [ADD] mail_discuss_channel_unread_sort: New module#1827
miquelalzanillas wants to merge 2 commits intoOCA:17.0from
APSL:17.0-add-mail_discuss_channel_unread_sort

Conversation

@miquelalzanillas
Copy link
Copy Markdown

This module changes the ordering of channels in the Discuss sidebar.

Unread channels are placed before read channels. Inside the unread group, channels are ordered by most recent activity so the latest channel with new messages is shown first.

Read channels keep the standard alphabetical ordering from Odoo.

@mpascuall @BernatObrador @ppyczko

@miquelalzanillas miquelalzanillas force-pushed the 17.0-add-mail_discuss_channel_unread_sort branch from cca0758 to 55c01b5 Compare March 17, 2026 22:08
@miquelalzanillas miquelalzanillas changed the title [ADD] mail_discuss_channel_unread_sort: New module [17.0] [ADD] mail_discuss_channel_unread_sort: New module Mar 17, 2026
Copy link
Copy Markdown

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Tested in runboat.

Copy link
Copy Markdown

@BernatObrador BernatObrador left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BernatObrador
Copy link
Copy Markdown

Just one thing, you added an extra commit [BOT] post-merge updates, please remove it.
Thanks!

Copy link
Copy Markdown

@peluko00 peluko00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Copy Markdown

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@miquelalzanillas
Copy link
Copy Markdown
Author

Hi @hbrunn ,

What do you think about this module? Can we merge it?

Thanks,

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants